-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ajout de la forme juridique au libelle du propriétaire #379
base: master
Are you sure you want to change the base?
Conversation
Merci Julien, @MaelREBOUX tu as pu faire un test ? |
SELECT dformjur, ddenom, rtrim(ddenom)||' ('||rtrim(dformjur)||')' AS nom
FROM cadastre.dgi_prop
WHERE dformjur IN
(
SELECT dformjur
FROM cadastre.dgi_prop
WHERE dformjur <> '' AND gtoper = '2'
GROUP BY dformjur
ORDER BY dformjur
)
LIMIT 100 donne
|
Bonjour Julien, Je ne suis pas du tout favorable à cette évolution sous cette forme car le résultat est une transformation du nom du propriétaire dans toute la base de données. C'est trop dangereux de systématiser et que ça se retrouve dans le relevé de propriété et sur les bordereaux. Si c'est une question de facilitation de l'identification du type de personnes morales, je l'entend parfaitement mais si le but c'est que ça apparaissent uniquement dans les interfaces de cadastrapp, il faut procéder autrement. |
Je viens de voir que la forme juridique est déjà dans la base, le champ existe déjà dans la table, on peut peut-être juste le changé côté requête java pour affichage côté client. Quel est le besoin exacte ? tu souhaiterais le voir afficher ou ? |
Le problème est que la forme juridique apparait certaines fois dans le ddenom, et pas d'autre... Par exemple pour les SCI, des fois on le retrouve devant le nom de l'entreprise, et d'autres fois on ne l'a pas. Une commune nous a fait remonter le besoin de voir apparaitre la forme juridique, après peu importe que ce soit entre parenthèse à coté du ddenom ou bien juste dans une colonne a coté. Il faut juste que ça remonte au niveau du propriétaire. |
Mais est-ce que c'est OK si ça apparaît juste dans l'IHM de cadastrapp et PAS dans les impressions "officielles" (BP RP et exports). |
A priori Ok si ça n'apparait que dans l'IHM, en revanche je pense qu'il serait judicieux dans les exports d'ajouter le champ dformjur lorsqu'il est disponible. Si jamais on a réellement un besoin au niveau des RP/BP, je réouvrirais une autre issue, mais à priori ce n'est pas le cas actuellement. |
OK. En ce cas ça me va si c'est circonscrit au niveau de l'IHM. Du coup @jusabatier : tu fermes cette PR je pense ? Pour mémo : liste des valeurs possible page 8 du document DGFIP. |
Je ferme la PR dès que le problème sera résolu, histoire d'en garder une trace jusque là. |
@pierrejego Est-ce que ça a été ajouté dans la v1.7 ? |
@jusabatier et bien non, mais je peux encore le faire on a pas fait la release de la 1.7. Par contre, ce n'est plus très clair pour moi, il faut que je modifie les services pour afficher uniquement à l'écran c'est bien ça. Et dans tous les écrans ou il y a ddenom ? Mais il ne faut pas d'impact sur les exports PDF ? |
Le problème était que nos communes n'ont pas accès à la forme juridique de l'entreprise. Apparement il y a un champ en BDD qui correspond à ça. L'idée était donc de faire remonter cette info dans la fiche d'info et les export CSV. |
Eventuellement créer une colonne supplementaire dans l'IHM (ce serait plus propre que directement dans le ddenom). Et pas de modification sur les BP/RP |
Tjs raccord avec mon dernier commentaire : #379 (comment) Donc : fermer cette PR : faire une issue et une nouvelle PR. |
@MaelREBOUX @catmorales @landryb vous en pensez quoi en 2022 ? |
Toujours pas favorable avec cette évolution qui dénaturerait des données super importantes (le nom des comptes prpprio). cf mon dernier commentaire : #379 (comment) @jusabatier si tu veux rendre cette info accessible à tes utilisateurs : tu peux le faire en base. |
Pour moi, je reste sur ce qui avait été dit dans les derniers commentaires :
@MaelREBOUX Je vois mal une secrétaire de mairie interroger la BDD, et forker cadastrapp juste pour ce usecase c'est infernal au niveau maintenance... |
Actuellement dans le cas d'une entreprise, la forme juridique n'apparait pas.
Ce patch permet de la faire apparaitre entre parenthèse dans le libelle.
Cette information nous a été demandé par l'une de nos communes.